-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MAINTENANCE] Added additional repository functional test instead of unit tests. #895
[MAINTENANCE] Added additional repository functional test instead of unit tests. #895
Conversation
Could you please bring this pull request up to date with master? Alternatively you can allow me to update your branch, then I can do it myself via GitHub most of the time. |
1bb19f1
to
d204b77
Compare
I did a rebase on the current master. |
d204b77
to
4b2b802
Compare
4b2b802
to
25a03e6
Compare
25a03e6
to
3f795a7
Compare
5ae4b1d
to
da3a347
Compare
@sebastian-meyer we have to remove the temporary file in TokenRepositoryTest.php:46, but according to Codacy |
Yes, please revert this back to |
da3a347
to
ef27570
Compare
Instead of unit testing repositories it is recommended to do functional tests.